Pull request overview

Adiciona o novo renderizador DanfeSimplificadoTipo2 (NT 2026.003, tpImp=6) para NF-e mod. 55 em layout “cupom” de largura fixa e altura dinâmica, composto por blocos empilhados (9 divisões) e com suíte de testes dedicada para validar cada divisão e cenários de contingência/altura.

Changes:

• Implementa DanfeSimplificadoTipo2 e os blocos de renderização SimplificadoTipo2/* cobrindo as 9 divisões da NT (incl. 2 vias em contingência pendente).
• Estende DanfeViewModel com TributosIbsCbs, OperacaoNaoPresencial, ValorPagoTotal e TrocoTotal, além de PendenteAutorizacao.
• Inclui especificação OpenSpec e testes MSTest (DanfeSimplificadoTipo2Tests) + referência no .csproj.

Reviewed changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Code review automatizado — DANFE Simplificado Tipo 2 (NT 2026.003)

Feature bem estruturada e isolada (um bloco por divisão, zero-regressão respeitada nos leiautes existentes, .csproj de produção é SDK-style então os arquivos novos entram por globbing, e o reuso de BlocoInformacaoFiscal é geometricamente compatível com a página de 280pt). Boa cobertura de teste por divisão + contingência + smoke de altura.

Deixei 7 comentários inline. Os de maior impacto, recomendados antes do merge:

1. Truncamento de infCpl/infAdFisco (≤160 chars) — risco de fidelidade fiscal (NT exige fidelidade ao XML).
2. QR Code em JPEG — usar PNG (lossless) + dispor o Bitmap.

Refinamentos menores (não bloqueiam): overflow silencioso da estimativa de altura, mutação do viewModel de input, corte do código do produto (>15 vs Substring(0,10)) + NRE potencial, campo público morto OperacaoNaoPresencial, e o parâmetro state ignorado em UrlNFeConsulta.

Triviais só de nota (não inline): a linha de homologação (bloco NFC reusado) centraliza em x=132.5 vs x=140 do restante (~7,5pt); Salvar usa SerializationModeEnum.Incremental num doc novo (Standard gera menor); e vale confirmar que .Formatar() fixa cultura pt-BR para os asserts "60,00"/"9,00" não quebrarem no CI.

Segurança: baixo risco (render de texto/imagem a partir do ViewModel; sem injeção/segredos/manipulação de path).

Revisão gerada com Claude Code.

Gate 8 — Revisão de Código (7 perspectivas paralelas)

Revisão automatizada por 7 perspectivas independentes (Arquitetura, Segurança, Performance, Testabilidade, Observabilidade, Resiliência, Conformidade de Linguagem), adaptadas para stack .NET.

Resultado consolidado: REJEITADO — não por veto direto de nenhuma perspectiva individual (todas as 7 aprovaram isoladamente), mas pelo critério de consolidação do gate: "Duas ou mais perspectivas encontram issues ALTO no mesmo componente". Isso ocorre em BlocoQrCodeT2.cs (linha ~39): a Perspectiva de Performance encontrou o vazamento de Bitmap (já também apontado no review anterior do @rhfranzoni) e a Perspectiva de Resiliência encontrou, de forma independente, que uma falha na geração do QR Code (ex.: string malformada) propaga sem tratamento e aborta a composição de TODO o restante do documento — inclusive a 2ª via obrigatória em contingência pendente ("Via do Estabelecimento"). A combinação dos dois problemas no mesmo bloco eleva o risco: um recurso não descartado + ausência de degradação graciosa no mesmo ponto do código.

Esta revisão já encontrou o review anterior de @rhfranzoni e os comentários do Copilot no PR — os comentários abaixo cobrem apenas achados net-new das 7 perspectivas que ainda não estavam registrados (não duplico o que já foi apontado sobre vazamento de Bitmap/JPEG, mutação do viewModel, inconsistência de corte do código do produto, campo OperacaoNaoPresencial morto, ou o parâmetro state ignorado em UrlNFeConsulta).

Nota importante, para transparência: os comentários já existentes de @rhfranzoni e do Copilot apontam pelo menos 2 achados reais que as 7 perspectivas desta rodada NÃO capturaram: (1) BlocoConsumidorT2.cs:28 — detecção de CNPJ/CPF via .Trim().Length == 14 falha se o documento vier formatado com pontuação (pode rotular CNPJ como CPF), e (2) DanfeViewModel.cs:338 — campo público OperacaoNaoPresencial sem nenhum uso no PR. Ambos seguem válidos e não resolvidos.

RESULTADO: REJEITADO (regra de consolidação, não veto individual) — recomenda-se resolver o ponto de BlocoQrCodeT2.cs (dispose do Bitmap + tratamento de falha isolando o QR do resto do documento) antes do merge. Os demais achados ALTO (cobertura de teste em ramos de truncamento) podem ir em follow-up, mas idealmente antes do próximo gate.

Revisão gerada com Claude Code (Gate 8 — revisao-codigo).